Skip to content

Conversation

@renaudhartert-db
Copy link
Contributor

What changes are proposed in this pull request?

This PR fixes a couple of typos in open_ai_client.py.

How is this tested?

N/A

@github-actions
Copy link

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-py

Inputs:

  • PR number: 829
  • Commit SHA: 5214a7a5f6ef683355adc8c43726ba7b80356d35

Checks will be approved automatically on success.

@eng-dev-ecosystem-bot
Copy link
Collaborator

Test Details: go/deco-tests/11934182985

@renaudhartert-db renaudhartert-db changed the title Fix typo [Internal] Fix a couple of typos in open_ai_client.py Nov 20, 2024
@renaudhartert-db renaudhartert-db added this pull request to the merge queue Nov 20, 2024
Merged via the queue into main with commit 74f9a4f Nov 20, 2024
19 of 20 checks passed
@renaudhartert-db renaudhartert-db deleted the renaud.hartert/typo branch November 20, 2024 14:05
tanmay-db added a commit that referenced this pull request Dec 10, 2024
### Bug Fixes

 * Update Changelog file ([#830](#830)).

### Internal Changes

 * Fix a couple of typos in open_ai_client.py ([#829](#829)).
 * Update SDK to OpenAPI spec ([#834](#834)).

### API Changes:

 * Added `databricks.sdk.service.cleanrooms` package.
 * Added `delete()` method for [w.aibi_dashboard_embedding_access_policy](https://databricks-sdk-py.readthedocs.io/en/latest/workspace/settings/aibi_dashboard_embedding_access_policy.html) workspace-level service.
 * Added `delete()` method for [w.aibi_dashboard_embedding_approved_domains](https://databricks-sdk-py.readthedocs.io/en/latest/workspace/settings/aibi_dashboard_embedding_approved_domains.html) workspace-level service.
 * Added `databricks_gcp_service_account` field for `databricks.sdk.service.catalog.CreateCredentialRequest`.
 * Added `databricks_gcp_service_account` field for `databricks.sdk.service.catalog.CredentialInfo`.
 * Added `gcp_options` field for `databricks.sdk.service.catalog.GenerateTemporaryServiceCredentialRequest`.
 * Added `databricks_gcp_service_account` field for `databricks.sdk.service.catalog.UpdateCredentialRequest`.
 * Added `cached_query_schema` field for `databricks.sdk.service.dashboards.QueryAttachment`.
 * Added .
 * Removed `gcp_service_account_key` field for `databricks.sdk.service.catalog.CreateCredentialRequest`.

OpenAPI SHA: 7016dcbf2e011459416cf408ce21143bcc4b3a25, Date: 2024-12-05
github-merge-queue bot pushed a commit that referenced this pull request Dec 11, 2024
### Bug Fixes

* Update Changelog file
([#830](#830)).


### Internal Changes

* Fix a couple of typos in open_ai_client.py
([#829](#829)).
* Update SDK to OpenAPI spec
([#834](#834)).


### API Changes:

 * Added `databricks.sdk.service.cleanrooms` package.
* Added `delete()` method for
[w.aibi_dashboard_embedding_access_policy](https://databricks-sdk-py.readthedocs.io/en/latest/workspace/settings/aibi_dashboard_embedding_access_policy.html)
workspace-level service.
* Added `delete()` method for
[w.aibi_dashboard_embedding_approved_domains](https://databricks-sdk-py.readthedocs.io/en/latest/workspace/settings/aibi_dashboard_embedding_approved_domains.html)
workspace-level service.
* Added `databricks_gcp_service_account` field for
`databricks.sdk.service.catalog.CreateCredentialRequest`.
* Added `databricks_gcp_service_account` field for
`databricks.sdk.service.catalog.CredentialInfo`.
* Added `gcp_options` field for
`databricks.sdk.service.catalog.GenerateTemporaryServiceCredentialRequest`.
* Added `databricks_gcp_service_account` field for
`databricks.sdk.service.catalog.UpdateCredentialRequest`.
* Added `cached_query_schema` field for
`databricks.sdk.service.dashboards.QueryAttachment`.
 * Added .
* Removed `gcp_service_account_key` field for
`databricks.sdk.service.catalog.CreateCredentialRequest`.

OpenAPI SHA: 7016dcbf2e011459416cf408ce21143bcc4b3a25, Date: 2024-12-05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants